Skip to content

Comments

Change CI to only be manually triggered to save on CI runs#252

Merged
kkraus14 merged 1 commit intoNVIDIA:mainfrom
kkraus14:manual_ci_triggering
May 14, 2025
Merged

Change CI to only be manually triggered to save on CI runs#252
kkraus14 merged 1 commit intoNVIDIA:mainfrom
kkraus14:manual_ci_triggering

Conversation

@kkraus14
Copy link
Contributor

Changes CI to require manually invoking it to save on CI runs. Previously, it would run on every commit on PRs regardless of if they were in draft or ready for review. We've used this configuration in the cuda-python repo and it hasn't been particularly painful and has allowed for more free flowing commits without burning CI resources unnecessarily.

@gmarkall
Copy link
Contributor

gmarkall commented May 14, 2025

Do we need to write the "ok to test" comment every time we want a CI run after this is merged?

@kkraus14
Copy link
Contributor Author

Do we need to write the "ok to test" comment every time we want a CI run after this is merged?

Yes

@gmarkall gmarkall added the 5 - Ready to merge Testing and reviews complete, ready to merge label May 14, 2025
@kkraus14 kkraus14 merged commit 773070f into NVIDIA:main May 14, 2025
37 checks passed
@isVoid isVoid mentioned this pull request May 21, 2025
isVoid added a commit that referenced this pull request May 21, 2025
- Allow External Code to Use Cooperative Group (#240)
- Improve debug info for kernel arguments (#242)
- Allow Numba NVRTC Binding Search Additional Paths (#254)
- Add Bfloat16 High Level API, Documentation (#245)
- add a test to use bf16 bindings inside device functions (#244)
- Change CI to only be manually triggered to save on CI runs (#252)
- Simplify the CI build and test matrix (#249)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

5 - Ready to merge Testing and reviews complete, ready to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants